Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

merge : 댓글,og 태그,모바일 반응형 구현 #37

Merged
merged 5 commits into from
Feb 29, 2024
Merged

Conversation

sheepdog13
Copy link
Owner

@sheepdog13 sheepdog13 commented Feb 29, 2024

PR 체크리스트

아래 항목을 확인해 주세요:

  • 커밋 메시지가 우리의 가이드라인을 따르고 있는지 확인하세요
  • 변경 사항에 대한 테스트가 추가되었는지 확인하세요 (버그 수정 / 기능 추가)
  • 문서가 추가되거나 업데이트되었는지 확인하세요 (버그 수정 / 기능 추가)

PR 유형

이 PR은 어떤 종류의 변경을 가져오나요?

  • 버그 수정
  • 새로운 기능 추가
  • 코드 스타일 업데이트 (서식, 로컬 변수)
  • 리팩터링 (기능 변경 없음, API 변경 없음)
  • 빌드 관련 변경
  • CI 관련 변경
  • 문서 내용 변경
  • 애플리케이션 / 인프라 변경
  • 기타... 설명:

관련 이슈

이슈 번호: #36

현재 동작은 무엇인가요?

모바일 반응형,기본 og,meta태그,댓글 기능이 없습니다.

새로운 동작은 무엇인가요?

  • 모바일 반응형을 max-width 400px때 부터 구현
  • helmet 라이브러리로 og,meta 태그 home, posting 따로 관리
  • giscus로 blogcomment 리포지토리의 GitHub Discussions을 활용해 댓글 구현

이 PR은 호환성 변경을 도입하나요?

  • 아니요

기타 정보

스크린샷 2024-02-29 오후 8 36 30 스크린샷 2024-02-29 오후 9 05 28

- giscus로 app의 title과 같은 Discussions이 있으면 댓글처럼 보여준다.
- type이 home일때는 블로그에 관한 meta 태그
- type이 posting일때는 posting에 관한 meta 태그 설정
@sheepdog13 sheepdog13 merged commit 643ff9b into master Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant